Repository navigation
fix(mm-ping-test): Use and schedule uefi based tests - #761
nicksinger wants to merge 1 commit into
Conversation
okurz
left a comment
okurz
left a comment
There was a problem hiding this comment.
@nicksinger please let the commit message state why we want this
| UEFI_PFLASH_CODE="$pflash_code" \ | ||
| UEFI_PFLASH_VARS="$pflash_vars" \ |
There was a problem hiding this comment.
@Martchus can you help to find the right settings so that we don't need to specify literal custom code+vars variables, e.g. just the cert+resolution setting?
There was a problem hiding this comment.
I will try with the settings from https://progress.opensuse.org/issues/198365?#note-10
There was a problem hiding this comment.
I managed to get rid of UEFI_PFLASH_CODE and replace it with an explicit UEFI_PFLASH_RESOLUTION, see https://github.com/os-autoinst/os-autoinst-scripts/compare/f98957d4955aa46db9a490a9c768c985ea3be7a1..52c22ff4597059e6d5a152fdee2d2ba4fe6d1f4b which results in a passing test: https://openqa.opensuse.org/tests/6268126. However, UEFI_PFLASH_VARS seems to be necessary to get the test running (see https://openqa.opensuse.org/tests/6268067 for a test with only UEFI_PFLASH_RESOLUTION).
5e5fb69 to
f98957d
Compare
done 👍 |
|
Just rebase to get rid of the style check failures |
| ${machine}: | ||
| backend: qemu | ||
| settings: | ||
| WORKER_CLASS: qemu_x86_64,tap |
There was a problem hiding this comment.
It would perhaps be cleaner to add the UEFI-related settings here in the machine definition (especially because the machine will now also be called uefi).
Or actually this still needs #763 |
done, thanks for the hint! |
There was a problem hiding this comment.
I had another look, so this is probably fine, see #761 (comment).
Probably you also don't need the resolution.
84875fd to
d906448
Compare
|
Re-requesting review since I moved around almost everything. VRs with the most recent changes are: |
The last 64bit snapshot or Tumbleweed is from August 2025. This change switches the test to uefi and therefore allows it to use a more recent snapshot. Related ticket: https://progress.opensuse.org/issues/207066
4 participants
Related ticket: https://progress.opensuse.org/issues/207066
Note: this requires an adjusted schedule from os-autoinst/os-autoinst-distri-opensuse#26988